Skip to content

DSA tag protocol support #93010

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

DSA tag protocol support #93010

wants to merge 12 commits into from
+351 −63

Conversation

yangbolu1991
Copy link
Member

@yangbolu1991 yangbolu1991 commented Jul 11, 2025 •
edited
Loading

This PR is to support DSA tag protocol in DSA core driver. Also added NXP NETC tag protocol driver, and enabled on NXP i.MX943 as an example to verify.

There is a hal dependency: zephyrproject-rtos/hal_nxp#575
Thanks a lot.

Attached dsa sample log.

[00:00:00.056,000] <inf> phy_mii: PHY (2) ID 20005CA2
[00:00:00.107,000] <inf> phy_mii: PHY (3) ID 20005CA2
[00:00:00.144,000] <inf> dsa_netc: DSA user port 0 Link down
[00:00:00.145,000] <inf> dsa_netc: DSA user port 1 Link down
[00:00:00.145,000] <inf> nxp_imx_eth_psi: ENETC3 MAC: 00:04:9f:00:11:00
*** Booting Zephyr OS build v4.2.0-rc2-120-gb0fc94da3d01 ***
[00:00:00.155,000] <inf> net_dsa_sample: [0] User interface 1 found.
[00:00:00.155,000] <inf> net_dsa_sample: [1] User interface 2 found.
[00:00:00.155,000] <inf> net_dsa_sample: DSA ports init - OK
[00:00:02.214,000] <inf> phy_mii: PHY (3) Link speed 100 Mb, full duplex
[00:00:02.214,000] <inf> dsa_netc: DSA user port 1 Link up
[00:00:02.214,000] <inf> net_dsa_sample: Port 2 is up
uart:~$
uart:~$
[00:00:10.174,000] <inf> net_dhcpv4: Received: 10.193.20.37
uart:~$
uart:~$
uart:~$
uart:~$ net ping 10.193.20.164
PING 10.193.20.164
28 bytes from 10.193.20.164 to 10.193.20.37: icmp_seq=1 ttl=64 time=0 ms
28 bytes from 10.193.20.164 to 10.193.20.37: icmp_seq=2 ttl=64 time=0 ms
28 bytes from 10.193.20.164 to 10.193.20.37: icmp_seq=3 ttl=64 time=0 ms
[00:00:23.722,000] <inf> phy_mii: PHY (3) is down
[00:00:23.722,000] <inf> dsa_netc: DSA user port 1 Link down
[00:00:28.163,000] <inf> phy_mii: PHY (2) Link speed 100 Mb, full duplex
[00:00:28.163,000] <inf> dsa_netc: DSA user port 0 Link up
[00:00:28.163,000] <inf> net_dsa_sample: Port 1 is up
uart:~$
uart:~$
uart:~$
[00:00:34.122,000] <inf> net_dhcpv4: Received: 10.193.20.163
uart:~$ net ping 10.193.20.164
PING 10.193.20.164
28 bytes from 10.193.20.164 to 10.193.20.163: icmp_seq=1 ttl=64 time=1 ms
28 bytes from 10.193.20.164 to 10.193.20.163: icmp_seq=2 ttl=64 time=0 ms
28 bytes from 10.193.20.164 to 10.193.20.163: icmp_seq=3 ttl=64 time=0 ms
uart:~$

@yangbolu1991
west.yml: update hal_nxp to support netc tag driver
6e85c56 
Updated hal_nxp to support netc tag driver.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 11, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_nxp zephyrproject-rtos/hal_nxp@7a52cbb (master) zephyrproject-rtos/hal_nxp#575 zephyrproject-rtos/hal_nxp#575/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_nxp DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Jul 11, 2025
@yangbolu1991
net: dsa: move DSA core options to subsystem
bfc2530 
Moved DSA core options to subsystem.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
drivers: ethernet: move NXP NETC DSA driver to dsa directory
29174d8 
Moved NXP NETC DSA driver to dsa directory.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
net: dsa: add tag protocol definition and binding
1ed3be9 
Added tag protocol definition and binding.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
net: dsa: add NXP NETC tag protocol driver
5ec0588 
Added NXP NETC tag protocol driver.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
net: dsa: support tag protocol setup
ab90049 
Supported tag protocol setup.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
drivers: eth_nxp_imx_netc: allow to transmit DSA tag frame
bc1c11f 
Allowed to transmit DSA tag frame.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
drivers: dsa_nxp_imx_netc: add tag protocol support
8f59cbb 
Added tag protocol support.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
dts: arm: nxp_imx943_m33: add NETC switch node
1a5c358 
Added NETC switch node.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
boards: nxp: imx943_evk: add NETC switch support for M33
5bd8bed 
Added NETC switch support for M33.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
samples: net: dsa: add imx943_evk_mimx94398_m33_ddr board support
0c3098e 
Added imx943_evk_mimx94398_m33_ddr board support.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
boards: nxp: imx943_evk: doc update for NETC switch support for M33
5a91672 
Added NETC switch support for M33 in doc.

Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com>
@yangbolu1991
Copy link
Member Author

Updated to fix twister issues.
Thanks.

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clamattia clamattia Awaiting requested review from clamattia

@dbaluta dbaluta Awaiting requested review from dbaluta

@decsny decsny Awaiting requested review from decsny

@dleach02 dleach02 Awaiting requested review from dleach02

@iuliana-prodan iuliana-prodan Awaiting requested review from iuliana-prodan

@JiafeiPan JiafeiPan Awaiting requested review from JiafeiPan

@jukkar jukkar Awaiting requested review from jukkar

@lmajewski lmajewski Awaiting requested review from lmajewski

@maass-hamburg maass-hamburg Awaiting requested review from maass-hamburg

@manuargue manuargue Awaiting requested review from manuargue

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@pdgendt pdgendt Awaiting requested review from pdgendt

@Raymond0225 Raymond0225 Awaiting requested review from Raymond0225

@rlubos rlubos Awaiting requested review from rlubos

@ssharks ssharks Awaiting requested review from ssharks

At least 2 approving reviews are required to merge this pull request.

Assignees

@jukkar jukkar

@rlubos rlubos

Labels
area: Devicetree area: Ethernet area: Networking area: Samples Samples DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_nxp platform: NXP Drivers NXP Semiconductors, drivers platform: NXP MPU
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@yangbolu1991 @jukkar @rlubos